Working with lesson_3_branch. by yaprogrammer18-yanchi · Pull Request #4 · yaprogrammer18-yanchi/C_Homework_

yaprogrammer18-yanchi · 2025-09-30T19:23:27Z

Добавила файлы домашней работы от урока о Компиляции. Удалила папку самого урока чтобы не путаться. Теперь все файлы лежат только в src.

WoWaster

Общее замечание: вынесите всю логику в отдельные функции. Ну и посмотрите, какие комментарии правда нужны.

WoWaster · 2025-10-09T09:19:14Z

src/bracketBalance.c

+
+int main()
+{
+    int* stackForBrackets = calloc(100, sizeof(int)); // сделаем большой стек состоящий из 100 нулей


Писать комментарии --- отдельное искусство.
Но для начала есть два простых правила:

Пишите комментарии над сущностью, которую комментируете.

Если комментарий описывает ровно то, что написано в коде --- он не нужен.

WoWaster · 2025-10-09T09:21:44Z

src/bracketBalance.c

+    scanf("%100[^\n]", stringWithBrackets); // ввели строку (огранчение 100 символов)
+
+    int count = 0; // счетчик того, сколько позиций занято в стеке
+    char str1 = ' '; // здесь будут символы из строки


Почему эту переменную нельзя объявить внутри цикла?

Название переменной странное. Во-первых, она типа char, а называет как будто это строка. Во-вторых, слишком общее, поэтому Вам и захотелось написать комментарий.

WoWaster · 2025-10-09T09:22:32Z

src/bracketBalance.c

+    char str2 = '(';
+    char str3 = ')';


Научитесь объявлять константы. Хотя здесь они ни к чему. У Вас str2 и str3 встречаются ниже в коде по одному разу.

WoWaster · 2025-10-09T09:23:36Z

src/bracketBalance.c

+
+        if (str1 == str2) // Ищем окрывающие скобки
+        {
+            stackForBrackets[count] = 2; // пусть 2 - это открывающая скобка


А если вдруг кто-то напишет в стек значение 42, что будет? Используйте максимально узкие типы. Здесь гораздо лучше бы подошел bool.

WoWaster · 2025-10-09T09:23:55Z

src/bracketBalance.c

+                count -= 1;
+                // этими действиями мы убрали закрытые скобки, будто их и не было
+            } else {
+


Тут какая-то лишняя строка

WoWaster · 2025-10-09T09:26:42Z

src/oneStringInAnother.c

+    {
+        bool flag = true; // Это значит, что все символы строк совпадают, но если хотя бы один символ будет отличаться, будем менять на 0 и выходить из цикла
+        for (int j = 0; j < lenForSmallerString; j++) {
+            if (listForBiggerString[i + j] != listForSmallerString[j]) {


А если i = lenForBiggerString-1, а j = lenForSmallerString? Какие данные мы будем читать?

Я так понимаю, что те, которые просто дальше лежат в памяти и уже никак не относятся к моей программе... Исправлю!

WoWaster · 2025-10-09T09:27:19Z

src/zerosInMassive.c

+    // пусть массив задан в программе
+    int listWithNumbers[] = { 0, 1, 3, 0, 6, 0, 7, 4, 6, 2, 0, 0 };
+    int countOfZeros = 0;
+    size_t lengthOfMassive = sizeof(listWithNumbers) / sizeof(int);


Massive --- это прилагательное! Array --- существительное.

WoWaster · 2025-10-09T09:29:17Z

src/oneStringInAnother.c

+    size_t lenForBiggerString = sizeof(listForBiggerString) / sizeof(char);
+    size_t lenForSmallerString = sizeof(listForSmallerString) / sizeof(char) - 1; // так как последний символ нулевой
+
+    int countingOfEntry = 0; // счетчик вхождений


entryCounter тогда уж. Counting обычно герундий

WoWaster · 2025-10-09T09:30:05Z

src/zerosInMassive.c

+int main()
+{
+    // пусть массив задан в программе
+    int listWithNumbers[] = { 0, 1, 3, 0, 6, 0, 7, 4, 6, 2, 0, 0 };


А список --- не массив.

WoWaster · 2025-10-09T09:31:04Z

src/zerosInMassive.c

+{
+    // пусть массив задан в программе
+    int listWithNumbers[] = { 0, 1, 3, 0, 6, 0, 7, 4, 6, 2, 0, 0 };
+    int countOfZeros = 0;


Либо numberOfZeroes, либо zeroCounter

WoWaster · 2025-10-11T09:05:26Z

src/bracketBalance.c

+{
+    //специальный счетчик, "(" это +1 ")" это -1. Если скобочная посл-ть правильная, counter == 0
+    int counter = 0;
+    bool bracketsExist = false;


Эта переменная правда нужна? Неужели счётчика не хватает?

WoWaster · 2025-10-11T09:18:19Z

src/bracketBalance.c

+    int counter = 0;
+    bool bracketsExist = false;
+
+    for (int i = 0; i < 100; i++) {


Почему строго 100? Неужели если в строке будет 345 символов всю остальную логику нельзя будет использовать?

WoWaster · 2025-10-11T09:19:55Z

src/zerosInMassive.c

+int main()
+{
+    // пусть массив задан в программе
+    int ArrayWithNumbers[] = { 0, 1, 3, 0, 6, 0, 7, 4, 6, 2, 0, 0 };


Почти :( тут с маленькой буквы

Эту задачу я зачту, а Вы поправите.

WoWaster · 2025-10-11T09:20:53Z

src/oneStringInAnother.c

+
+    int entryCounter = 0;
+    // i - индекс эл-та с которого будем рассматривать большую строку
+    for (unsigned int i = 0; i < lenForBiggerString; i++) {


Suggested change

for (unsigned int i = 0; i < lenForBiggerString; i++) {

for (unsigned i = 0; i < lenForBiggerString; i++) {

по стайлгайду

WoWaster · 2025-10-11T09:22:10Z

src/oneStringInAnother.c

+    for (unsigned int i = 0; i < lenForBiggerString; i++) {
+        bool flag = true;
+        for (unsigned int j = 0; j < lenForSmallerString; j++) {
+            if (i + j < lenForBiggerString) {


Это можно сделать проще. Рассматривайте i не от 0 до lenForBiggerString - 1, а меньше, тогда будет меньше проблем с границами.

…Homework_ into lesson_3_branch

yaprogrammer18-yanchi · 2025-10-12T18:04:03Z

Я изменила сообщение коммита и мне теперь выдает Merge branch 'lesson_3_branch' of github.com:yaprogrammer18-yanchi/C_Homework_ into lesson_3_branch... Можно я сделаю force push? :D

WoWaster

Осталось отформатировать... :(

WoWaster · 2025-11-14T13:28:26Z

И в своих ветках делайте force-push сколько хотите.

WoWaster

Я не понял, почему у Вас раздублировались файлы

WoWaster · 2025-12-06T10:43:58Z

src/CompilationProcess/bracketBalance.c

Зачтено.

WoWaster · 2025-12-06T10:45:15Z

src/CompilationProcess/oneStringInAnother.c

Пусть уже будет.

yaprogrammer18-yanchi added 3 commits September 28, 2025 19:16

Added hw for 3rd lesson about compilation

1f90b71

Add three files from homework (brackets, zeros and strings)

733067a

Delete dir with useless files in order not to be messed up

9729fb8

yaprogrammer18-yanchi requested a review from WoWaster September 30, 2025 19:24

yaprogrammer18-yanchi added 2 commits October 4, 2025 17:28

Remove mistakes from files of hw

ee719f3

Rename and remove files

0b92d9b

WoWaster requested changes Oct 9, 2025

View reviewed changes

Fix mistakes in all hw

0ae9722

WoWaster requested changes Oct 11, 2025

View reviewed changes

yaprogrammer18-yanchi added 5 commits October 11, 2025 19:49

Fix mistakes in zeros and strings, still think about bracket balance

cc9c73c

Fix bracket balance. Now you can put as many symbols as you want

d2a3a74

Fix brackets balancegit add oneStringInAnother.c !

972ebc4

Fix bracket balance again

4fb61a2

Merge branch 'lesson_3_branch' of github.com:yaprogrammer18-yanchi/C_…

4bf6e87

…Homework_ into lesson_3_branch

yaprogrammer18-yanchi added 2 commits October 14, 2025 00:15

Finish working with bracketBalance.c

922b6ec

Use clang-format for bracketBalance.c

13568ea

WoWaster requested changes Nov 14, 2025

View reviewed changes

yaprogrammer18-yanchi added 5 commits November 22, 2025 21:38

add clang-format config

fafb969

add clang-format config just need to commit it one more time

b6f3179

formatted files

c03d67a

Add cmake

e17da74

Add Cmake

b85c2f8

WoWaster approved these changes Dec 6, 2025

View reviewed changes

	for (unsigned int i = 0; i < lenForBiggerString; i++) {
	for (unsigned i = 0; i < lenForBiggerString; i++) {

Comments

Conversation

yaprogrammer18-yanchi commented Sep 30, 2025

Uh oh!

WoWaster left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

yaprogrammer18-yanchi commented Oct 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

WoWaster left a comment

Choose a reason for hiding this comment

Uh oh!

WoWaster commented Nov 14, 2025

Uh oh!

WoWaster left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

yaprogrammer18-yanchi commented Oct 12, 2025 •

edited

Loading